Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ENV][Fix] #1 : auto-merge 정상적으로 동작안되는 문제 수정 #180

Merged
merged 1 commit into from
Nov 17, 2024

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 16, 2024

📝 PR 개요

auto-merge 정상적으로 동작안되는 문제 수정

🔍 변경 사항

  • "확인 요청" 상태에서 Approve가 2명 혹은 3명일 시 자동으로 "확인 요청" 라벨 삭제하고, "작업 완료" 라벨 붙인 후 머지될 수 있도록 수정하였습니다.
  • 기존 코드에서 에러가 났던 부분은 각 step에서의 return 관련된 부분임을 확인을 했고 이를 수정하였습니다.

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 16, 2024
@effozen
Copy link
Collaborator Author

effozen commented Nov 16, 2024

Action 특성상 Main 브렌치에 직접 업데이트되는 PR입니다.
이에 따라 본 코드는 깃허브 위에서 직접 편집이 되었습니다. (로컬의 그 어떠한 코드도 반영되어 있지 않습니다.)

@effozen effozen requested a review from leedongyull November 16, 2024 21:01
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!!

@juwon5272
Copy link
Collaborator

늦은 시간까지 고생하셨습니다!

Copy link
Collaborator

@leedongyull leedongyull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!!

@effozen effozen merged commit a7e2d1f into main Nov 17, 2024
4 of 8 checks passed
@effozen effozen deleted the feature/env/#1-auto-merge-bug-fix branch November 17, 2024 06:24
@effozen effozen added 작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 17, 2024
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 17, 2024
@effozen effozen removed the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants